Conversation
30892dd to
07e77c9
Compare
casaroli
left a comment
There was a problem hiding this comment.
gcc-multilib is only required if you are running ubuntu 64-bit and want to run native-sim 32-bit.
-
on 32-bit systems, there is no
gcc-multilibpackages -
on 64-bit only systems like arm64, there is no
gcc-multilibalso.
So, even that we support that, to add it to the github workflows would require more work (in the Dockefile), so I belive it is better if we just document this board, but not test it.
.devcontainer/zephyr/Dockerfile
Outdated
| dfu-util \ | ||
| file \ | ||
| gcc \ | ||
| gcc-multilib \ |
There was a problem hiding this comment.
this would break arm64 support as gcc-multilib is not available there.
| gcc-multilib \ |
.github/workflows/zephyr.yml
Outdated
| board: | ||
| - pico_plus2/rp2350b/m33 | ||
| - native_sim/native/64 | ||
| - native_sim |
There was a problem hiding this comment.
so we don't build in the CI as we will not have gcc-multilib
| - native_sim |
.github/workflows/zephyr.yml
Outdated
| matrix: | ||
| board: | ||
| - native_sim/native/64 | ||
| - native_sim |
There was a problem hiding this comment.
| - native_sim |
docs/boards/zephyr/native_sim.md
Outdated
| # 64-bit | ||
| west build -p always -b native_sim/native/64 src/samples/supervisor/zephyr/ | ||
|
|
||
| # 32-bit | ||
| west build -p always -b native_sim src/samples/supervisor/zephyr/ |
There was a problem hiding this comment.
| # 64-bit | |
| west build -p always -b native_sim/native/64 src/samples/supervisor/zephyr/ | |
| # 32-bit | |
| west build -p always -b native_sim src/samples/supervisor/zephyr/ | |
| west build -p always -b native_sim/native/64 src/samples/supervisor/zephyr/ |
There was a problem hiding this comment.
"Alternatively, the native_sim board could be used for supported 32-bit platforms."
docs/boards/zephyr/native_sim.md
Outdated
| - **supervisor** | ||
|
|
||
| **Note:** native_sim on 32-bit is not currently supported. | ||
| **Note:** The 32-bit variant requires `gcc-multilib` to be installed on your host machine (see [Prerequisites](#prerequisites)). |
There was a problem hiding this comment.
| **Note:** The 32-bit variant requires `gcc-multilib` to be installed on your host machine (see [Prerequisites](#prerequisites)). | |
| **Note:** The 32-bit variant requires `gcc-multilib` to be installed on your host machine (see [Prerequisites](#prerequisites)) if you are running a 64-bit system. |
docs/boards/zephyr/native_sim.md
Outdated
|
|
||
| Start with [Getting Started with Zephyr](../../GetStartedZephyr.md) to get a working build environment. | ||
|
|
||
| To build for the 32-bit `native_sim` variant, install `gcc-multilib` on your host machine: |
There was a problem hiding this comment.
| To build for the 32-bit `native_sim` variant, install `gcc-multilib` on your host machine: | |
| To build for the 32-bit `native_sim` variant on a 64-bit system, install `gcc-multilib` on your host machine: |
| ```bash | ||
| sudo apt install gcc-multilib | ||
| ``` | ||
|
|
There was a problem hiding this comment.
| Note: this is not required for 32-bit systems. |
docs/boards/zephyr/native_sim.md
Outdated
| ## Limitations | ||
|
|
||
| - **64-bit only**: 32-bit native_sim is not supported | ||
| - **32-bit requires gcc-multilib**: Install `gcc-multilib` on your machine before building the 32-bit variant |
There was a problem hiding this comment.
| - **32-bit requires gcc-multilib**: Install `gcc-multilib` on your machine before building the 32-bit variant | |
| - **32-bit requires gcc-multilib on 64-byt systems**: Install `gcc-multilib` on your machine before building the 32-bit variant |
07e77c9 to
75b3b25
Compare
Signed-off-by: SorinO <sorin.olari2@gmail.com>
…orrect ISA variant, 32 or 64 bit Signed-off-by: SorinO <sorin.olari2@gmail.com>
75b3b25 to
0e59c43
Compare
…o malloc was replaced with alligned_alloc Signed-off-by: SorinO <sorin.olari2@gmail.com>
…ation Signed-off-by: SorinO <sorin.olari2@gmail.com>
…FIG_OCRE_SHARED_HEAP_BUF_VIRTUAL with CONFIG_OCRE_SHARED_HEAP_BUF_SIZE Signed-off-by: SorinO <sorin.olari2@gmail.com>
Signed-off-by: SorinO <sorin.olari2@gmail.com>
0e59c43 to
c06da1f
Compare
Signed-off-by: SorinO <sorin.olari2@gmail.com>
Description
This PR includes the following changes:
native_simfor mini, demo and supervisor samplesnative_simAlso, this PR fixes the following bugs:
added a check for
TARGET_ISA X86_64inwamr.cmaketo know if application is built for 32 or 64 bit architecturein
memory.cmalloc() was replaced with alligned_alloc(), because WAMR expects linear memory to be aligned before allocationfixed a bug in
wamr.cwith wrong parameter passed to user_malloc() functionFixes issue # native_sim #175
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
Tested mini, demo and supervisor for native_sim, native_sim/native/64 and b_u585i_iot02a
Checklist: